Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cookie의 path를 /auth에서 /로 변경 및 RefreshToken TTL설정 #610

Merged
merged 2 commits into from
Sep 14, 2023

Conversation

jeomxon
Copy link
Collaborator

@jeomxon jeomxon commented Sep 13, 2023

🔥 연관 이슈

close: #608

📝 작업 요약

  • cookie에 path를 /auth에서 /로 변경했습니다.
  • 레디스에서 RefreshToken의 TTL을 14일로 설정했습니다.

⏰ 소요 시간

1분

🔎 작업 상세 설명

/auth로 경로를 설정하게 되면 새로고침 시 쿠키가 삭제됩니다.
다른 경로에서 한번 받아오는 것은 되지만 그 이후에는 새로고침을 하면서 경로가 다르다고 판단하여 삭제하는 것 같아요.
/로 설정해야 새로고침을 해도 쿠키가 남아있습니다.

🌟 논의 사항

없음.

@github-actions
Copy link

github-actions bot commented Sep 13, 2023

Test Results

236 tests   236 ✔️  33s ⏱️
  61 suites      0 💤
  61 files        0

Results for commit 6b31325.

♻️ This comment has been updated with latest results.

@woo-chang woo-chang removed this from the 5차 데모데이 milestone Sep 14, 2023
@jeomxon jeomxon changed the title cookie의 path를 /auth에서 /로 변경 cookie의 path를 /auth에서 /로 변경 및 RefreshToken TTL설정 Sep 14, 2023
Copy link
Collaborator

@aiaiaiai1 aiaiaiai1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Collaborator

@woo-chang woo-chang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다 👍👍

Copy link
Collaborator

@tjdtls690 tjdtls690 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅡ 문 신 ㅡ

@jeomxon jeomxon merged commit 5ace436 into dev Sep 14, 2023
3 checks passed
@jeomxon jeomxon deleted the feat/#608 branch September 14, 2023 05:13
Gilpop8663 added a commit that referenced this pull request Sep 14, 2023

* fix: (#598) 로그인 시 쿠키를 주고 받을 수 있도록 fetch 설정 변경 (#600)

* cookie의 path를 /auth에서 /로 변경 및 RefreshToken TTL설정 (#610)

* refactor: (#608) cookie의 path를 루트 경로로 지정

* refactor: (#608) refreshtoken을 redis에 저장할 때 ttl설정

* 구글 애널리틱스 작동을 위한 모듈 설치, 게시글 본문의 링크 클릭 가능하도록 구현 (#613)

* fix: (#612) react-gtm-module을 react-ga4 로 대체, RouteChangeTracker 컴포넌트 추가

* feat: (#606) 게시글 작성 시 링크 넣기 기능 추가

* feat: (#606) 게시글 본문에 링크 있으면 a 태그로 인식되도록 구현

* chore: (#606) 공지사항 설명 수정

* chore: (#606) 불필요한 주석 삭제

* 로그인, 투표 통계, 회원정보 페이지 lazy import 적용 (#571)

* feat: (#556) lazy import 설정 및 트리쉐이킹 설정

* feat: (#556) 로그인, 투표 통계, 회원정보 입력 페이지 lazy import 적용

* feat: (#556) 번들 이름 매번 바뀌도록 변경 및 수정되었던 코드 복구

* feat: (#556) Suspense 코드 복구

---------

Co-authored-by: jero_kang <81199414+inyeong-kang@users.noreply.github.com>

* 게시글 작성 시 이미지 파일을 보낼 때 webp 로 압축하여 성능 개선 (#614)

* feat: (#555) browser-image-compression 설치 및 본문 이미지 훅에 적용

* feat: (#555) 선택지 옵션 사진을 webp로 변환하도록 구현

---------

Co-authored-by: 최우창 <clllickme@naver.com>
Co-authored-by: Jun-Hyeok Sin <tjdtls690@gmail.com>
Co-authored-by: jero_kang <81199414+inyeong-kang@users.noreply.github.com>
Co-authored-by: jeomxon <jeomxon@gmail.com>
Co-authored-by: aiaiaiai1 <cvc100326@naver.com>
Co-authored-by: chsua <113416448+chsua@users.noreply.github.com>
Co-authored-by: lookh <103165859+aiaiaiai1@users.noreply.github.com>
Co-authored-by: 김영길/KIM YOUNG GIL <Gilpop8663@users.noreply.github.com>
Co-authored-by: jero_kang <inyeong-kang@users.noreply.github.com>
Co-authored-by: chsua <csj1919@naver.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] 쿠키 path 지정 후 새로고침 시 쿠키가 사라지는 문제
4 participants